Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Namespace: Add pallet to have namespace feature #560

Merged
merged 4 commits into from
Jan 9, 2025

Conversation

vatsa287
Copy link
Member

@vatsa287 vatsa287 commented Jan 6, 2025

This PR implements Namespace as a concept on chain.

@amarts amarts merged commit cd868be into dhiway:develop Jan 9, 2025
3 checks passed
smohan-dw added a commit that referenced this pull request Jan 26, 2025
* Registries: Remove registry-id as a argument for create (#559)

Signed-off-by: Shreevatsa N <[email protected]>

* pallet-asset: Added new test-case for DistributionLimitExceeded. (#522)

This issue has been structured  similarly to the `asset_create_should_succeed` test.
This test will ensure that an asset can be issued successfully and will include coverage for reaching and exceeding the `MaxAssetDistribution` limit.

- Test Initialization:
The test sets up a space, approves it, and creates an asset similarly to the `asset_create_should_succeed` test.
- Asset Issuance:
The loop issues the asset up to the `MaxAssetDistribution` limit (set to 25 for testing). Each iteration simulates issuing the asset to a different recipient.
- Error Handling:
After the limit is reached, the test tries to issue the asset to one more recipient, which should trigger the `DistributionLimitExceeded` error. This ensures that the pallet's logic correctly handles and enforces the distribution limit.

This implementation checks that the asset issuance process is valid up to the `MaxAssetDistribution`  limit and correctly throws an error when the `limit` is exceeded.

* Namespace: Add pallet to have namespace feature (#560)


Signed-off-by: Shreevatsa N <[email protected]>

* namespace: Remove approval requirement for usage (#563)

Signed-off-by: Shreevatsa N <[email protected]>

* docs: Update readme for PR validation requirements. (#540)

* network-score :  Add Test Cases for SpaceMismatch error

Fixes: #304 
* network-score: Add Test Cases for SpaceMismatch Error in pallet-network-score

Signed-off-by: Aman Raj <[email protected]>

* namespace: Add digest and blob for namespace create 

Fixes: #564

Signed-off-by: Shreevatsa N <[email protected]>

* namespace: Identifier updates (#565)

Signed-off-by: Shreevatsa N <[email protected]>

* namespace: Integrate with Registries framework

Signed-off-by: Shreevatsa N <[email protected]>

* registries: Integration testing with Namespace

Signed-off-by: Shreevatsa N <[email protected]>

* registries: Benchmarking updates for Namespace

Signed-off-by: Shreevatsa N <[email protected]>

* entries: Integration testing with updated Registries & Namespace

Signed-off-by: Shreevatsa N <[email protected]>

* Update to use new identifiers

Signed-off-by: Shreevatsa N <[email protected]>

* namespace, registries: Update lookup table on create (#569)

Signed-off-by: Shreevatsa N <[email protected]>

* Update the CORD version to 0.9.5 (#570)

Signed-off-by: Shreevatsa N <[email protected]>

* Registries: Add tests for AuthorizationNotFound (#566)

* registries: add tests for AuthorizationNotFound
---------

Co-authored-by: Mudit Sarda <[email protected]>

* Fixes: #310 Add test cases for SchemaNotFound error (#551)

Signed-off-by: Vaibhav <[email protected]>

* fix upstream merge conflicts

---------

Signed-off-by: Shreevatsa N <[email protected]>
Signed-off-by: Aman Raj <[email protected]>
Signed-off-by: Vaibhav <[email protected]>
Co-authored-by: Shree Vatsa N <[email protected]>
Co-authored-by: RITANKAR SAHA <[email protected]>
Co-authored-by: Aman Raj <[email protected]>
Co-authored-by: muditDhiway <[email protected]>
Co-authored-by: Mudit Sarda <[email protected]>
Co-authored-by: VAIBHAV SAHU <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants